Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Android composer padding: Director's Cut #7412

Merged
merged 3 commits into from
Jan 9, 2025

Conversation

mozzius
Copy link
Member

@mozzius mozzius commented Jan 9, 2025

Supercedes #7228

As far as I can tell, KeyboardAvoidingView is adding too much padding when the keyboard is closed on Android 15. We add a negative offset to the KeyboardAvoidingView, then remove the containing padding when the keyboard is open - seems to do the trick. We already do something similar for iOS.

Button nav

Before Before After After
Before Before After After

Gesture nav

Before Before After After
Before Before After After

Android 11 (unchanged)

Before Before After After

Web is unchanged, since this all hinges on safe area insets which don't exist on web

Test plan

Test the various combinations of Android version and Navigation mode, with keyboard open and closed

@arcalinea arcalinea temporarily deployed to samuel/android-15-composer-2 - social-app PR #7412 January 9, 2025 17:14 — with Render Destroyed
Copy link

github-actions bot commented Jan 9, 2025

Old size New size Diff
6.86 MB 6.86 MB -105 B (-0.00%)

@arcalinea arcalinea temporarily deployed to samuel/android-15-composer-2 - social-app PR #7412 January 9, 2025 17:23 — with Render Destroyed
@mozzius mozzius merged commit 3b9a51c into main Jan 9, 2025
7 checks passed
@mozzius mozzius deleted the samuel/android-15-composer-2 branch January 9, 2025 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants